New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add SiFive InclusiveCache #128
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but I'd like to add a more complex config to CI (just needs to build).
Merging for now with knowing that BOOM is for some reason broken on this bump. Will address later. |
I think we should file issues about things that we will address later. I filed one about my L2 config complaint, and you should file one about fixing boom with the l2, IMO. Otherwise its very easy to lose track of things. |
Issues or it never happened. |
Thanks to @jerryz123 this: riscv-boom/riscv-boom#315 should address the issue. |
No description provided.